Skip to content

chore: gitignore the strix quick gate's local scratch artifact directory - #1453

Merged
seonghobae merged 1 commit into
mainfrom
fix/gitignore-strix-runs-artifact
Aug 31, 2026
Merged

chore: gitignore the strix quick gate's local scratch artifact directory#1453
seonghobae merged 1 commit into
mainfrom
fix/gitignore-strix-runs-artifact

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Summary

  • scripts/ci/strix_quick_gate.sh writes its report/log artifacts to $REPO_ROOT/strix_runs (ARTIFACT_REPORTS_DIR) when run outside an ephemeral CI workspace — e.g. running scripts/ci/test_strix_quick_gate.sh locally against a persistent checkout leaves strix_runs/ behind as an untracked directory.
  • Add strix_runs/ to .gitignore, alongside the existing runtime/cache entries (__pycache__/, .coverage, .pytest_cache/, .codegraph/).

Test plan

  • python3 -m pytest tests/ -q — 1897 passed, 1 skipped, 21 subtests passed

Generated by Claude Code

scripts/ci/strix_quick_gate.sh writes its report/log artifacts to
$REPO_ROOT/strix_runs when run outside an ephemeral CI workspace (e.g.
scripts/ci/test_strix_quick_gate.sh run locally), leaving it as an
untracked directory in the checkout.
@seonghobae
seonghobae marked this pull request as ready for review August 30, 2026 12:47
@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 7 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ab004efa-7291-4a29-960c-50df87f2b26d

📥 Commits

Reviewing files that changed from the base of the PR and between 1d8e872 and aec1122.

📒 Files selected for processing (1)
  • .gitignore

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

Copy link
Copy Markdown
Contributor Author

The failing opencode-review check here is the same "no current-head verdict yet" gate we've seen on other PRs today — it's not something in this PR's one-line .gitignore diff. The scheduler has already run (scan-pr-queue succeeded) but review dispatch is rate-limited to a small budget per run across the whole .github PR queue (currently ~25 open PRs waiting on review), so this one hasn't gotten its dispatch slot yet. No opencode-agent review exists for this head SHA; Devin's bot review (informational, no issues found) isn't a substitute.

This will resolve on its own via the scheduler's normal rotation — no fix needed on this PR, just queue rotation. Keeping it watched.


Generated by Claude Code

@seonghobae
seonghobae merged commit 2461b82 into main Aug 31, 2026
61 of 79 checks passed
@seonghobae
seonghobae deleted the fix/gitignore-strix-runs-artifact branch August 31, 2026 01:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants